Skip to content

Update PHP and firebase/php-jwt version requirements#229

Merged
rfeiner merged 4 commits intomessagebird:masterfrom
bobvandevijver:patch-1
Feb 19, 2026
Merged

Update PHP and firebase/php-jwt version requirements#229
rfeiner merged 4 commits intomessagebird:masterfrom
bobvandevijver:patch-1

Conversation

@bobvandevijver
Copy link
Contributor

@bobvandevijver bobvandevijver commented Feb 18, 2026

Only allow firebase/php-jwt to resolve GHSA-2x45-7fc3-mxwq. Also bumps PHP to >= 8.0 (Support for PHP 7 has been discontinued since 03 Nov 2022).

Resolves #227, #228.

Only allow firebase/php-jwt to resolve GHSA-2x45-7fc3-mxwq. Also drop duplicate version constraints for PHP (>=7.2 is also valid for any PHP 8 or later versions).
@frederikbosch
Copy link

PHP JWT uses ^8.0 fir PHP dep so composer.json of this package might/should drop PHP7 too.

@bobvandevijver
Copy link
Contributor Author

PHP JWT uses ^8.0 fir PHP dep so composer.json of this package might/should drop PHP7 too.

Hmm, did not want to make that decision for this PR, but it does make sense indeed.

Also, support for PHP 7 has been discontinued since 03 Nov 2022.
@bobvandevijver bobvandevijver changed the title Update firebase/php-jwt version requirements Update PHP and firebase/php-jwt version requirements Feb 18, 2026
@WoBBeLnl
Copy link

Is there any way we can expedite the release of a new version with this PR? I am not using Github often (only for private projects) so I don't know how to approve the waiting approval or is this something that can only be done by maintainers (if so, can we tag them?)

@bobvandevijver
Copy link
Contributor Author

It's in the message:

This workflow requires approval from a maintainer.

I prefer to not tag maintainers so soon after PR creation.

@frederikbosch
Copy link

But this security advisory is having big implications: people cannot install their packages because of it.

@frederikbosch
Copy link

So if @rfeiner could follow this up, that would be great.

@WoBBeLnl
Copy link

I submitted a support ticket to Bird linking to the issue and this PR.

@yoeriboven
Copy link

@rfeiner @ErikBooijMB Could you guys take a look? This is a security concern.

- Replace Docker-based Psalm action (bundles PHP 7.4) with setup-php
- Drop PHP 7.4 from test matrix
- Set phpVersion in psalm.xml for PHP 8.0+ analysis

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rfeiner rfeiner merged commit 9e804c0 into messagebird:master Feb 19, 2026
7 checks passed
@bobvandevijver bobvandevijver deleted the patch-1 branch February 19, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add firebase/php-jwt 7 to composer

6 participants

Comments